Enable ODBC Build On Linux by alinaliBQ · Pull Request #160 · Bit-Quill/arrow

alinaliBQ · 2026-03-11T22:46:30Z

Implements basic Linux build and unicode support is enabled. Tested locally and verified the driver can connect to DBT Labs via isql.

Enable Linux ODBC build with unicode support
Added docker-compose for Flight SQL ODBC
Register ODBC after build
Replaced boost::lexicographical_compare with std::lexicographical_compare
Fixed conversion bugs in SetAttributeSQLWCHAR

justing-bq · 2026-03-13T17:19:47Z

      - name: Register Flight SQL ODBC Driver
        run: |
-          sudo cpp/src/arrow/flight/sql/odbc/install/mac/install_odbc.sh $(pwd)/build/cpp/${{ matrix.build-type }}/libarrow_flight_sql_odbc.dylib
+          sudo cpp/src/arrow/flight/sql/odbc/install/unix/install_odbc.sh $(pwd)/build/cpp/${{ matrix.build-type }}/libarrow_flight_sql_odbc.dylib


Don't we want this same "Register Flight SQL ODBC Driver" section for linux as well?

This section is inside compose.yaml:

arrow/compose.yaml

Lines 531 to 535 in 2afc5fd

command: >

/bin/bash -c "

/arrow/ci/scripts/cpp_build.sh /arrow /build &&

sudo /arrow/cpp/src/arrow/flight/sql/odbc/install/unix/install_odbc.sh /usr/local/lib/libarrow_flight_sql_odbc.so &&

/arrow/ci/scripts/cpp_test.sh /arrow /build"

Because Linux docker file has the test sessions etc inside the docker command, I added ODBC registration at the same place

Next: resolve the string conversion issues in tests. Write simple helper function to convert wstring (or wchar_t*) to `std::vector<SQLWCHAR>` should solve the issue. Disable ODBC test build on Linux - Building Linux test in ODBC will be in a separate PR - Reducing dockerfile will be in a separate PR Resolve sqlucode.h build issue * Doesn't impact macOS. Check CI if it impacts Windows ODBC. Resolve `sign-compare` warnings - Need to have same type during comparison. - Now `Built target arrow-odbc-spi-impl-test` is successful. Add forward function header Resolve system.cc build errors Still have some wsigns stuff to fix later. In-progress fix with unicode issues on system_dsn.cc TODO - still need to make the `ToSqlWCharPtr` function safe. Fix Linux build errors with configuration.cc * Finish fixing wchar related build errors at configuration.cc * Still have errors at `system_dsn.cc` Remove some log messages This partially reverts commit 568f72d. All tests on macOS passing locally. In-progress fix read SQLWCHAR string In-progress Fix ODBC Build errors on Linux - Mar2 week - Add `[[fallthrough]];` to indicate intentional fallthrough. - Add `default` handling case. - Add casts for `record_number` compares Attempt to fix Windows build from odbc_statement.cc header change In-progress Fix ODBC Build errors on Linux - remove `std::move(options);` - Remove `boost` library install and add boost::headers to cmake lists, which resolves a lot of the boost issues I think - Continue on Monday. Add wrappers for `reinterpret_cast<LPCWSTR>` This was causing an issue on Linux Replace `boost::lexicographical_compare` with `std::lexicographical_compare` Restore boost usages and add `BOOST_NO_CXX98_FUNCTION_BASE` Undo Set build level warning to production It didn't resolve the build errors related to boost on Linux (or any other build errors) Set build level warning to production * If this acts as a workaround for boost, then we don't have to redo boost In-progress Fix the `LPCWSTR` convert error on Linux - Plus formatting fix - Plus build fix for casting More boost disables I think the solution would be to replace boost completely. Now the build works on macOS. Continue in-progress fix odbc build Temp boost disables In-progress fix ODBC build - Add `unixodbc-dev` to dockerfile - There are changes to macOS static build PR that I will need to incorporate back to `apache-odbc` later Enable ODBC mac and Windows in case of errors Enable ODBC build on Linux Add ODBC=ON to dockerfile Fix `arm64` cannot found error from typo Add `DOCKER_VOLUME_PREFIX` and other environment settings Add Linux docker to `compose.yaml` Disable mac and Windows ODBC for faster build More disable Draft ODBC Linux implementation Comment out non-ODBC items

* remove ODBC dockerfile as it is not needed. * in-progress of register ODBC * move unix odbc registration script to common place

* resolve the issue of ODBC APIs not exported on Linux

Trying to figure out why `Set(FlightSqlConnection::DISABLE_CERTIFICATE_VERIFICATION` just cannot pass default values. Somehow Add configuration.cc logs In-progress implement unicode support conversion [In-progress] One change that (somehow) enables DBTLabs Flight SQL connection - using `ENVID` as `static constexpr` enabled the environmentid variable to be read properly. Then other custom keys like `abc` and `Description` also started to work. Fixes the issue of strings too

alinaliBQ · 2026-03-19T17:04:58Z

@@ -40,7 +40,16 @@ if [ ! -f "$ODBC_64BIT" ]; then
  exit 1
 fi



@justing-bq this is the unix registration script

alinaliBQ · 2026-03-19T17:09:46Z

+    command: >
+      /bin/bash -c "
+        /arrow/ci/scripts/cpp_build.sh /arrow /build &&
+        sudo /arrow/cpp/src/arrow/flight/sql/odbc/install/unix/install_odbc.sh /usr/local/lib/libarrow_flight_sql_odbc.so &&
+        /arrow/ci/scripts/cpp_test.sh /arrow /build"


ODBC build, registration, and test is here

alinaliBQ · 2026-03-19T17:13:50Z


+// flight_sql_connection.h needs to be included first due to conflicts with windows.h
+#include "arrow/flight/sql/odbc/odbc_impl/flight_sql_connection.h"


For unicode support on Linux

alinaliBQ · 2026-03-19T17:15:28Z

+  // Read keys before reading DSN. This is a workaround to unixodbc driver manager
+  // unable to read DSN keys


todo for myself change the comment

alinaliBQ · 2026-03-19T17:16:15Z


 #pragma once

+#define BOOST_NO_CXX98_FUNCTION_BASE  // ARROW-17805


needed for build on Linux

alinaliBQ · 2026-03-19T17:19:33Z

  }

  // Reset GetData value offsets.
-  if (num_binding_ != get_data_offsets_.size() && reset_get_data_) {


todo use ~~const cast~~ c++ style cast

alinaliBQ · 2026-03-19T17:21:08Z

  EXPECT_FALSE(std::wstring(message).empty());
 }

-// TODO: verify that `SQLGetConnectOption` is not required by Excel.


removed comment that should have been removed before

alinaliBQ · 2026-03-19T17:22:08Z

  endif()
-
  add_test_case(${REL_TEST_NAME}


todo undo this change

alinaliBQ closed this Mar 11, 2026

alinaliBQ reopened this Mar 11, 2026

alinaliBQ force-pushed the linux-ci-build-new branch from d4e5463 to 8ac65b3 Compare March 12, 2026 19:26

alinaliBQ marked this pull request as ready for review March 12, 2026 19:33

alinaliBQ force-pushed the linux-ci-build-new branch from 8ac65b3 to 2afc5fd Compare March 12, 2026 19:35

justing-bq reviewed Mar 13, 2026

View reviewed changes

alinaliBQ force-pushed the apache-odbc branch from cc41aae to 767d612 Compare March 13, 2026 18:21

alinaliBQ added 9 commits March 13, 2026 11:22

Reuse existing docker image (no install of unixodbc)

0e114ba

Install unixodbc-dev inside docker

f157cac

Clean up Linux changes

68c1d62

Comments for next TODO

489d57e

Use ubuntu-cpp-odbc compose command and change to release mode

330f68a

* remove ODBC dockerfile as it is not needed. * in-progress of register ODBC * move unix odbc registration script to common place

Export ODBC APIs on Linux

de27e42

* resolve the issue of ODBC APIs not exported on Linux

Clean up ODBC code

49e53df

Use reinterpret_cast as Linux wchar_t is char16

126bbe7

alinaliBQ force-pushed the linux-ci-build-new branch from 2afc5fd to 126bbe7 Compare March 13, 2026 18:47

alinaliBQ changed the title ~~Enable ODBC Build On Linux without unicode support~~ Enable ODBC Build On Linux Mar 18, 2026

alinaliBQ commented Mar 19, 2026

View reviewed changes

justing-bq approved these changes Mar 19, 2026

View reviewed changes

alinaliBQ and others added 2 commits March 19, 2026 11:54

Address my own PR comments

16c29c7

Add patch for Windows build

308adde

alinaliBQ merged commit 17e312a into apache-odbc Mar 19, 2026
32 of 40 checks passed

Navigation Menu

Uh oh!

Enable ODBC Build On Linux - #160

Enable ODBC Build On Linux#160
alinaliBQ merged 12 commits into
apache-odbcfrom
linux-ci-build-new

alinaliBQ commented Mar 11, 2026 •

edited

Loading

Uh oh!

justing-bq Mar 13, 2026

Uh oh!

alinaliBQ Mar 13, 2026

Uh oh!

alinaliBQ Mar 19, 2026 •

edited

Loading

Uh oh!

alinaliBQ Mar 19, 2026

Uh oh!

alinaliBQ Mar 19, 2026

Uh oh!

alinaliBQ Mar 19, 2026

Uh oh!

alinaliBQ Mar 19, 2026

Uh oh!

alinaliBQ Mar 19, 2026

Uh oh!

alinaliBQ Mar 19, 2026 •

edited

Loading

Uh oh!

alinaliBQ Mar 19, 2026

Uh oh!

alinaliBQ Mar 19, 2026

Uh oh!

alinaliBQ Mar 19, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

	command: >
	/bin/bash -c "
	/arrow/ci/scripts/cpp_build.sh /arrow /build &&
	sudo /arrow/cpp/src/arrow/flight/sql/odbc/install/unix/install_odbc.sh /usr/local/lib/libarrow_flight_sql_odbc.so &&
	/arrow/ci/scripts/cpp_test.sh /arrow /build"


		// flight_sql_connection.h needs to be included first due to conflicts with windows.h
		#include "arrow/flight/sql/odbc/odbc_impl/flight_sql_connection.h"

		// Read keys before reading DSN. This is a workaround to unixodbc driver manager
		// unable to read DSN keys


		#pragma once

		#define BOOST_NO_CXX98_FUNCTION_BASE // ARROW-17805

Uh oh!

Conversation

alinaliBQ commented Mar 11, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

alinaliBQ Mar 19, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

alinaliBQ Mar 19, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

alinaliBQ commented Mar 11, 2026 •

edited

Loading

alinaliBQ Mar 19, 2026 •

edited

Loading

alinaliBQ Mar 19, 2026 •

edited

Loading